-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CCD multi-track capability for Princeton cameras #6
base: master
Are you sure you want to change the base?
Conversation
…ate pull request.
…rack class previously used for ADAndor. Updated picam library - added 32-bit library version that Princeton now have available. Corrected some printf string parameter errors. Enhanced error detection and handling behaviour. Added Linux build capability (was previously missing), 64-bit only. This requires ftp://ftp.princetoninstruments.com/Public/Software/Official/PICam/Picam_SDK.run installed. Tested end-to-end (on Windows only) using a ProEM camera.
The use cases are quite different. _RBV is used to indicate what the hardware currently is using. It does not modify the output record. asyn:READBACK updates the output record based on the readback. I have typically only used asyn:READBACK in specialized cases, for example the FileNumber in the file plugins, which can be autoincremented by the driver, or manually set by the user. For most parameters I don't like the driver to modify the output record. Having the output and readback be different can give the user an indication that something is wrong, whereas updating the output record could hide this problem.
I don't think many people are using ADPICam, so I think you will probably find a number of issues like this that have not been addressed yet. It would be great if you can test and improve the driver! I am currently using ADLightField rather than ADPICam, but ADLightField has its own issues, and I would be happy to know that ADPICam was a viable alternative. |
… than by enumeration order).
The current software has a capability for selecting a camera (by it's enumeration order) from the AvailableCameras_RBV record. It's more useful to select a specifc camera by it's serial number. The AvailableCameras_RBV record is populated (with camera model) through the readEnum method. I've prepended the camera serial number to the enum string. I don't understand the cause of this reported error: |
width/sizeX and height/sizeY variable names were used in an inconsistent way. Included other completed pull requests.
Added Linux SDK files. Added (optional) build using Acton spectrometer.
I would like to query a couple of issues:
a. Is there any policy regarding use of asyn:READBACK, as opposed to use of _RBV records?
b. The st.cmd file does not currently select a specific camera (it will select the default camera 0, i.e. whichever is the first detected).
This is an issue because camera settings are loaded from the hardware on start-up.